Closed Bug 1808904 Opened 2 years ago Closed 2 years ago

Unblocking a dll from the upper part of the about:third-party page (before the page is fully updated) causes other dll's to be unblocked as well

Categories

(Firefox :: Launcher Process, defect)

Firefox 110
All
Windows
defect

Tracking

()

VERIFIED FIXED
110 Branch
Tracking Status
firefox-esr102 --- unaffected
firefox108 --- unaffected
firefox109 --- unaffected
firefox110 + verified

People

(Reporter: emilghitta, Assigned: gstoll)

References

(Blocks 1 open bug, Regression)

Details

(Keywords: regression)

Attachments

(3 files)

Found in

  • Firefox 110.0a1 (BuildId:20230106095123)

Affected versions

  • Firefox 110.0a1 (BuildId:20230106095123)

Tested platforms

  • Affected platforms: All Windows.
  • Unaffected platforms: macOS & ubuntu.

Preconditions

Steps to reproduce

  1. Access the about:third-party page.
  2. Hit CTRL + O.
  3. Close the File Explorer and refresh the about:third-party page.
  4. Block the TortoiseGit.dll.
  5. Click on "Restart Nightly now" when prompted.
  6. Close the File Explorer and refresh the about:third-party page.
  7. Block the drivefsext.dll.
  8. Click on "Restart Nightly now" when prompted.
  9. Unblock the drivefsext.dll from the top of the about:third-party page.
  10. Click on "Restart Nightly now" when prompted.
  11. Hit CTRL + O.
  12. Close the File Explorer and refresh the about:third-party page.

Expected result

  • Only the drivefsext.dll was unblocked.

Actual result

  • Both drivefsext.dll and the TortoiseGit.dll were unblocked.

Regression range
I suspect that the fix for Bug 1808727 may have introduced this regression. (I can't reproduce with a build before that patch).

I think that the following scenario is similar to the one described in this bug (probably the root cause is the same):

  1. Access the about:third-party page.
  2. Hit CTRL + O.
  3. Close the File Explorer and refresh the about:third-party page.
  4. Block the TortoiseGit.dll.
  5. Click on "Restart Nightly now" when prompted.
  6. Close the File Explorer and refresh the about:third-party page.
  7. Unblock the TestDllBlocklist_UserBlocked.dll.
  8. Click on "Restart Nightly now" when prompted.

Same as the issue described above, the other dll's are getting unblocked (TortoiseGit.dll). The testDllBlocklist_UserBlocked.dll is still blocked (as it should). But this should not cause TortoiseGit to get unblocked.

:gstoll, since you are the author of the regressor, bug 1808727, could you take a look?

For more information, please visit auto_nag documentation.

Flags: needinfo?(gstoll)

I don't think this is actually caused by the fix to bug 1808727, but fixing that may have exposed this problem. Anyway, nice catch! I'll take a look.

Assignee: nobody → gstoll
Status: NEW → ASSIGNED
Flags: needinfo?(gstoll)

I believe this is timing-related - if you click to unblock a module before the background task finishes, the list of blocked modules won't be populated yet and so we will write an incomplete list to the blocklist file. I haven't been able to reproduce this on my system but believe I have a fix.

Pushed by gstoll@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/8de779164240 don't allow blocking modules before we've loaded the full list r=yjuglaret
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 110 Branch

This issue is verified fixed using Firefox 110.0a1 (BuildId:20230109212101) on Windows 10 64bit & Windows 11 64bit.

Previously blocked dlls cannot be unblocked before the dll list was loaded (The unblock button is disabled). The symptoms described in the bug description and in comment 1 (other dlls are wrongly unblocked) are no longer reproducible if unblocking the dlls after the list was loaded and the unblock button becomes active.

Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: